Skip to content

[QI-3367] Allow specifiying query tags as a dict upon connection creation#749

Merged
sreekanth-db merged 2 commits intodatabricks:mainfrom
jiabin-hu:query_tag2
Mar 9, 2026
Merged

[QI-3367] Allow specifiying query tags as a dict upon connection creation#749
sreekanth-db merged 2 commits intodatabricks:mainfrom
jiabin-hu:query_tag2

Conversation

@jiabin-hu
Copy link
Contributor

@jiabin-hu jiabin-hu commented Mar 4, 2026

What type of PR is this?

  • Refactor
  • Feature
  • Bug Fix
  • Other

Description

For session level query tags,

  1. adding the option to specify session query tags as a dict rather than a serialized string, to be consistent w/ statement level
  2. if specified both as dict param, and serialized string as session param, dict param takes precedence, overwriting everything in session param

How is this tested?

  • Unit tests
  • E2E Tests
  • Manually
  • N/A

Related Tickets & Documents

Signed-off-by: Jiabin Hu <jiabin.hu@databricks.com>
Signed-off-by: Jiabin Hu <jiabin.hu@databricks.com>
Copy link
Contributor

@sreekanth-db sreekanth-db left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@sreekanth-db sreekanth-db requested a review from gopalldb March 5, 2026 07:06
@sreekanth-db sreekanth-db merged commit e916f71 into databricks:main Mar 9, 2026
34 of 36 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants